-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement subscribe poll for the devicesim #248
Implement subscribe poll for the devicesim #248
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When I ran this - it gives the POLL response every 5 seconds showing Dublin, but when went and changed the TZ value in another terminal to Paris, the POLL event continued giving the old Dublin value. I'd expect it to start transmitting the updated value. Even after restarting the POLL it still displays the old value "Dublin"
Thanks for catching the error. I will take a look to see what is going on. |
f757f6c
to
a990b55
Compare
f935229
to
4c52090
Compare
I addressed your comment and I think this part is ready to merge. Whenever you get a chance, please test it and see will you get the expected output for subscribe poll and once. Thanks |
4c52090
to
296ae54
Compare
ca3b876
to
7b9b4fd
Compare
7b9b4fd
to
e991962
Compare
|
||
|
||
RUN cd $GOPATH/src/github.com/onosproject/onos-config/tools/test/devicesim/gnmi && go install | ||
RUN cd ./gnmi_target && go install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This gives me a
# _/home/devicesim/gnmi_target
./gnmi_utils.go:115:1: syntax error: non-declaration statement outside function body
1- The devsim supports subscribe poll as well.
2- Porting the gnmi server code to the repo that we add the new features that we need.